🤖🤖🤖 Update quality-playbook skill to v1.5.6 + add agent#1402
Conversation
🔍 Skill Validator Results
Summary
Full validator output```text Found 1 skill(s) [quality-playbook] 📊 quality-playbook: 66,332 BPE tokens [chars/4: 73,092] (comprehensive ✗), 128 sections, 44 code blocks [quality-playbook] ⚠ Skill is 66,332 BPE tokens (chars/4 estimate: 73,092) — "comprehensive" skills hurt performance by 2.9pp on average. Consider splitting into 2–3 focused skills. Found 1 agent(s) Validated 1 agent(s)✅ All checks passed (1 skill(s), 1 agent(s)) |
a2b9013 to
57eae8d
Compare
|
Oops, sorry about that. Good catch -- this PR ships both the skill and the
agent, but the agent should refer to the skill in awesome-copilot, not my
github. The quality-playbook agent is a thin wrapper around the skill, with
a guardrail telling the user to install the skill if it's missing. If they
installed the agent from awesome-copilot, it should send them back to
awesome-copilot to install the skill as well, not to my github page.
I'm just finishing up a new version of the skill with a lot of
improvements. Is it better to update this PR or close it out and create a
new one?
…On Mon, Apr 27, 2026 at 9:20 PM Aaron Powell ***@***.***> wrote:
***@***.**** requested changes on this pull request.
------------------------------
On skills/quality-playbook/quality_gate.sh
<#1402 (comment)>
:
This should live within the scripts folder per the skills spec
<https://agentskills.io/specification>
------------------------------
On skills/quality-playbook/quality_gate.sh
<#1402 (comment)>
:
This file shouldn't have to do directory walking to find the SKILL.md
file, that should be always ../ from the location of the bash file
------------------------------
In agents/quality-playbook.agent.md
<#1402 (comment)>
:
> +
+You are a quality engineering orchestrator. Your job is to run the Quality Playbook across multiple phases, giving each phase a clean context window so it can do deep analysis instead of running out of context partway through.
+
+## Setup: find the skill
+
+Check that the quality playbook skill is installed. Look for SKILL.md in these locations, in order:
+
+1. `.github/skills/quality-playbook/SKILL.md` (Copilot)
+2. `.github/skills/SKILL.md` (Copilot, flat layout)
+3. `.claude/skills/quality-playbook/SKILL.md` (Claude Code)
+
+Also check for a `references/` directory alongside SKILL.md containing iteration.md, review_protocols.md, spec_audit.md, and verification.md.
+
+**If the skill is not installed**, tell the user:
+
+> The quality playbook skill isn't installed in this repository yet. Install it from the [quality-playbook repository](https://github.com/andrewstellman/quality-playbook):
What is the reason for "installing" this repo? Generally, we try to avoid
having agents/skills clone external repos as that results in opacity in the
security supply chain of what you install.
—
Reply to this email directly, view it on GitHub
<#1402 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABZLBCPVW763BQJZDSJ65ND4YABMBAVCNFSM6AAAAACX2KNBZWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DCOBVGE3DAOJWGQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
fine to iterate on this PR. We squash merge so "messy" history isn't really a problem |
- SKILL.md updated to v1.5.6 (2738 lines, was 2322 at v1.4.2). Banner + version stamps + JSON schema examples all bumped. - references/ expanded to 16 files (was 12). New: challenge_gate.md, code-only-mode.md, orchestrator_protocol.md, run_state_schema.md. Existing files updated to current QPB content. - phase_prompts/ added (9 files). New v1.5.x directory carrying externalized phase prompts the runner uses. - agents/ added inside the skill bundle (3 files: quality-playbook.agent.md, quality-playbook-claude.agent.md, calibration_orchestrator.md). The v1.5.6 install bundle includes these so README's Step 4 'claude --agent agents/...' invocation resolves from the install destination. - bin/citation_verifier.py added — required by quality_gate.py for byte-equality citation verification (per QPB BUG-005 closure). - quality_gate.sh retired in v1.4.5; replaced with quality_gate.py (Python rewrite with json.load + 108-test unit suite at upstream). agents/quality-playbook.agent.md (top-level awesome-copilot agent file) install instructions rewritten to install from awesome-copilot's own skills/quality-playbook/ folder rather than from the andrewstellman/ quality-playbook upstream — adopters discovering the skill via awesome-copilot can install without leaving the marketplace. Locations list expanded to all four supported tools (Copilot, Cursor, Claude Code, Continue). Pointer to upstream preserved as alternative for users who want the full script-driven install UX (auto-detect, marker directory creation, smoke checks). Closes github#1402.
…nce) PR github#1402 validator flagged 'name: "Quality Playbook"' as failing two rules: - Agent name must match filename (expected 'Quality Playbook.agent.md') - Agent name must be lowercase alphanumeric and hyphens only Change name to 'quality-playbook' in both the top-level agent file and the bundle copy under skills/quality-playbook/agents/. The bundle's quality-playbook-claude.agent.md was already correct.
bedbe84 to
ce0a291
Compare
Rebuilds branch from upstream/staged (was previously merged from upstream/main, which brought in materialized plugin files that fail Check Plugin Structure on PRs targeting staged). Changes vs. staged: - Update skills/quality-playbook/ to v1.5.6 (31 bundled assets: SKILL.md + LICENSE.txt + 16 references/ + 9 phase_prompts/ + 3 agents/ + bin/citation_verifier.py + quality_gate.py). - Add agents/quality-playbook.agent.md (top-level orchestrator). name: quality-playbook (validator-compliant). - Update docs/README.skills.md quality-playbook row description + bundled-assets list to v1.5.6. - Fix 'unparseable' → 'unparsable' in quality_gate.py (5 instances; codespell preference, both spellings valid). Closes the v1.4.0 → v1.5.6 update in a single clean commit on top of upstream/staged. The preserved backup branch backup-bedbe84-pre-rebuild (SHA bedbe84) holds the prior history for reference.
ce0a291 to
9f06edd
Compare
|
Force-pushed to rebuild this branch on top of upstream/staged instead of the previously-merged upstream/main. The earlier merge (commit 8553ea7) brought in materialized plugin files that fail Check Plugin Structure on staged-targeted PRs — those are auto-copied to main at publish time and shouldn't appear in the diff for staged. |
|
I'll merge this in as-is, but it's worth considering the validator feedback for a future iteration. The skill itself is big, there's a lot of tokens that it'll (potentially) use in what it does, which could be detrimental to performance and behaviour. |
|
Agreed, it was really valuable that the validator flagged that. I've got it
queued for 1.5.7:
https://github.com/andrewstellman/quality-playbook/blob/1.5.7/docs/design/QPB_v1.5.7_Design.md
(see #5). Expect a PR for that in the not too distant future.
…On Sun, May 10, 2026 at 9:32 PM Aaron Powell ***@***.***> wrote:
*aaronpowell* left a comment (github/awesome-copilot#1402)
<#1402 (comment)>
I'll merge this in as-is, but it's worth considering the validator
feedback for a future iteration.
The skill itself is *big*, there's a lot of tokens that it'll
(potentially) use in what it does, which could be detrimental to
performance and behaviour.
—
Reply to this email directly, view it on GitHub
<#1402 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABZLBCMPDGLCJKALXUAEEST42EUTVAVCNFSM6AAAAACX2KNBZWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DIMJWHE2DMMBVGY>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Summary
Changes from 1.3.x to 1.4.2:
agents/quality-playbook.agent.md— checks skill installation, explains phases, handles "keep going" / "help" / "what happened"Updated to v1.5.6.
Changes since v1.4.2:
SKILL.md modernized (skills bundle now includes phase_prompts/, agents/, and bin/citation_verifier.py per the v1.5.6 install contract; quality_gate.sh retired in v1.4.5 and replaced by quality_gate.py).
references/ expanded from 12 to 16 files.
The top-level agents/quality-playbook.agent.md now installs from awesome-copilot's own skills/quality-playbook/ folder instead of pointing users to the upstream Quality Playbook repo, so adopters discovering the skill via awesome-copilot can install without leaving the marketplace. Manual cp recipes provided for all four supported AI tools (Copilot, Cursor, Claude Code, Continue). Pointer to the upstream repo preserved as alternative for users who want the full script-driven install UX.
Empirically validated against Cursor, Claude Code, and GitHub Copilot install flows on 2026-05-08.
See: https://www.oreilly.com/radar/ai-is-writing-our-code-faster-than-we-can-verify-it/